Skip to content

Conversation

thet
Copy link
Member

@thet thet commented Oct 27, 2022

This is a non breaking change.

Adding the compiled bundle to npm allows to directly include it via a
CDN like so:

https://cdn.jsdelivr.net/npm/@patternslib/patternslib@9.7.0-alpha.2/dist/bundle.min.js
or
https://unpkg.com/@patternslib/patternslib@9.7.0-alpha.2/dist/bundle.min.js

We have to compile the bundle with the correct version BEFORE the npm
package is created with release-it. The next version is queried using
the LEVEL parameter (set by the different release targets) and the
"semver" tool.

@thet thet requested a review from jensens October 27, 2022 01:20
@thet thet force-pushed the add-dist branch 4 times, most recently from 6ade9af to b09bf1e Compare November 14, 2022 10:59
Copy link
Member

@jensens jensens left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I am probably the wrong person to review this, but it looks good as far as I can tell.

thet added 2 commits November 15, 2022 22:34
Creating a npm release with the Makefile prepares now a bundle build
with the correct package version added as a comment to the bundle files.
The compiled bundle can then be uploaded to npm if it is part of the git
repository (included by .gitignore) or part of the npm release
(preferred, included by .npmignore).

Adding the compiled bundle to npm allows to directly include it via a
CDN like so:

https://cdn.jsdelivr.net/npm/@patternslib/patternslib@9.7.0-alpha.2/dist/bundle.min.js
or
https://unpkg.com/@patternslib/patternslib@9.7.0-alpha.2/dist/bundle.min.js

We have to compile the bundle with the correct version BEFORE the npm
package is created with release-it. The next version is queried using
the LEVEL parameter (set by the different release targets) and the
"semver" tool.
@thet thet merged commit b7879d2 into main Nov 15, 2022
@thet thet deleted the add-dist branch November 15, 2022 21:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants